Skip to content

refactor(args.rs): make the flags id slices and the default values as a const - #2733

Merged
extrawurst merged 2 commits into
gitui-org:masterfrom
anas-contribs:refactor/constants
Oct 28, 2025
Merged

extrawurst merged 2 commits into
gitui-org:masterfrom
anas-contribs:refactor/constants

Conversation

@0x61nas

@0x61nas 0x61nas commented Oct 6, 2025 •

Copy link
Copy Markdown
Contributor

Refactor the string slices for the flag ids and the default values tow an explicit constants to improve the readability and the maintainability

I followed the checklist:

  • I added unittests (no need)
  • I ran make check without errors
  • I tested the overall application
  • I added an appropriate item to the changelog (no need)

extrawurst

This comment was marked as duplicate.

@extrawurst extrawurst added this to the v0.28 milestone Oct 28, 2025

@extrawurst extrawurst left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@extrawurst
extrawurst merged commit ec65b37 into gitui-org:master Oct 28, 2025
22 checks passed
cruessler pushed a commit to cruessler/gitui that referenced this pull request Oct 29, 2025
… a const (gitui-org#2733)

Co-authored-by: extrawurst <776816+extrawurst@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants